Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Review] Request from 'schubi2' @ 'yast/yast-autoinstallation/review_160113_bug_955878_merged_with_sp1' #179

Closed
wants to merge 5 commits into from

Conversation

schubi2
Copy link
Member

@schubi2 schubi2 commented Jan 13, 2016

Please review the following changes:

@kobliha
Copy link
Member

kobliha commented Jan 13, 2016

I'm afraid that this is wrong as well as the previous PR was. We want to ignore cobbler completely.
We don't want to show error or warning on this. But this is just my POV, maybe customers/PM/PjM want something different.

@schubi2
Copy link
Member Author

schubi2 commented Jan 14, 2016

I think espl. in Master we should go that way in order to get rid of the unused cobbler entry in the future .

@mvidner
Copy link
Member

mvidner commented Mar 18, 2016

This was replaced by #181.

@mvidner mvidner closed this Mar 18, 2016
@schubi2 schubi2 deleted the review_160113_bug_955878_merged_with_sp1 branch August 17, 2016 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants